Skip to content
This repository has been archived by the owner on Nov 4, 2018. It is now read-only.

Add redux boilerplate #25

Closed
wants to merge 27 commits into from
Closed

Add redux boilerplate #25

wants to merge 27 commits into from

Conversation

sorrycc
Copy link
Contributor

@sorrycc sorrycc commented Apr 11, 2016

Close #24

Usage

$ antd-init
? Please select boilerplate type (Use arrow keys)
❯ plain react - for simple project
  redux - for complex project

or

antd-init --type redux

@sorrycc
Copy link
Contributor Author

sorrycc commented Apr 11, 2016

@afc163 @yiminghe @benjycui 看下有没有问题。

afc163
afc163 reviewed Apr 11, 2016
View reviewed changes
boilerplates/redux/src/services/api.js
import fetch from 'isomorphic-fetch';

export async function getPlatforms() {
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

是不是搞一个能跑起来的接口？

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

有什么现成的接口建议么?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

http://ant.design/components/select/#demo-search-box

淘宝搜索。

@sorrycc
Copy link
Contributor Author

sorrycc commented Apr 11, 2016

https://github.com/ant-design/antd-init/blob/add-redux-boilerplate/boilerplates/redux/README.md
新脚手架的介绍文档。

@sorrycc
Copy link
Contributor Author

sorrycc commented Apr 12, 2016

api 部分已替换为可执行代码。

afc163
afc163 reviewed Apr 17, 2016
View reviewed changes
boilerplates/redux/README.md

> 后缀为 `.module.less` 或 `.module.css` 的会被解析为 css-modules

内置支持 jsx-control-statements
Copy link
Member

@afc163 afc163 Apr 17, 2016 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

个人不推荐用这个，太像传统模板了。

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

嗯，不要内置吧

@afc163
Copy link
Member

afc163 commented Apr 17, 2016

sorrycc/react-redux-boilerplate#1

afc163
afc163 reviewed Apr 17, 2016
View reviewed changes
boilerplates/redux/README.md

支持 css-modules

> 后缀为 `.module.less` 或 `.module.css` 的会被解析为 css-modules
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

需要做这个后缀区分么？

@benjycui benjycui mentioned this pull request Apr 18, 2016
Closed
@yiminghe
Copy link
Contributor

好复杂，项目的 readme 补充下下 --type 参数吧

@sorrycc
Copy link
Contributor Author

sorrycc commented Apr 18, 2016 •
edited
Loading

TODO：

  • 项目的 readme 补充下下 --type 参数
  • 示例换成 TODO LIST 的
  • 移除内置的 jsx-control-statements
  • Layout
  • Upgrade to antd@1
  • Fetch wrapper
  • Dynamic router
  • i18n
  • 默认 css module，不用 .module.less 的后缀 (需 ignore 掉 node_modules 下的 css 文件)
  • saga hmr
  • storyboard
  • README (snippets, ...)

@xyzget
Copy link

xyzget commented May 12, 2016

这个功能没有集成进去么？

@sorrycc
Copy link
Contributor Author

sorrycc commented May 12, 2016 •
edited
Loading

@xyzget 这周休假了，我下周搞完他。

@sorrycc
Copy link
Contributor Author

sorrycc commented May 18, 2016

有冲突，已手动 merge 到 master 。

@sorrycc sorrycc closed this May 18, 2016
@sorrycc
Copy link
Contributor Author

sorrycc commented May 18, 2016

Demo 换成 TODO App：

@sorrycc sorrycc deleted the add-redux-boilerplate branch May 18, 2016 09:55
@soda-x
Copy link
Contributor

soda-x commented May 18, 2016

人工点赞

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@sorrycc @afc163 @yiminghe @xyzget @soda-x